-
-
Notifications
You must be signed in to change notification settings - Fork 2.6k
New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
Fix issue #12928 (Report documentation looks a bit odd when there is no docs for a report) #22739
base: 5.x-dev
Are you sure you want to change the base?
Conversation
… there is no docs for a report)
I corrected what you asked for! Thanks for the help @michalkleiner! Could you review this PR again, please? |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Translation string is needed
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Can you also please check the PHPCS lint errors and correct those?
Fixes #12928 |
Lint errors corrected! :) |
Hi @michalkleiner ! Anything else to correct? |
Hey @michalkleiner , I'm sorry for bothering you, but are there any more changes to be made? If there are, I'll work on them right away, otherwise, could you merge this? Thank you so more for your time! |
Description:
This change addresses the issue where the report documentation appears odd or empty when no documentation is available for a report. Specifically, it ensures that a default message ("No documentation available for this report.") is displayed when reportDocumentation is empty or missing. This improves the user experience by providing clear feedback instead of showing a blank or incomplete UI.
Review